Skip to content

fix links internos (info: )#481

Merged
joaquinelio merged 1 commit intojavascript-tutorial:masterfrom
joaquinelio:webada
Feb 8, 2021
Merged

fix links internos (info: )#481
joaquinelio merged 1 commit intojavascript-tutorial:masterfrom
joaquinelio:webada

Conversation

@joaquinelio
Copy link
Copy Markdown
Member

No description provided.

@joaquinelio
Copy link
Copy Markdown
Member Author

Los links internos [ ] ( info : ... # ... ) necesitan un anchor al cual ir, faltaban en el repo inglés.
sin ellos encuentran el artículo pero no el item

Los 5 archivos target modificados no cambian su apariencia,
pero los archivos donde estan los links que los llamaban ahora pueden encontrarlos

Los probé con server local. no se puede probar en github.
Para probar online, los articulos y nros de linea de origen arreglados son:

https://es.javascript.info/nullish-coalescing-operator
1 2 12, 64
(info:logical-operators#or-finds-the-first-truthy-value)

https://es.javascript.info/array-methods
1 5 5, 460
(info:string#correct-comparisons)

https://es.javascript.info/rest-parameters-spread
1 6 2, 230
(info:object-copy#cloning-and-merging-object-assign)

https://es.javascript.info/extend-natives
1 9 5, 77
(info:static-properties-methods#statics-and-inheritance)

https://es.javascript.info/generators
1 12 1, 157
(info:rest-parameters-spread#spread-syntax))

@joaquinelio
Copy link
Copy Markdown
Member Author

Estos links no son tan importantes como los externos que ccorrijo en el pr posterior

porque el al cliquear va el capitulo correcto, solo que no salta al renglon referenciado

Copy link
Copy Markdown
Contributor

@vplentinax vplentinax left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK

@joaquinelio joaquinelio merged commit f7700b3 into javascript-tutorial:master Feb 8, 2021
@joaquinelio joaquinelio deleted the webada branch February 8, 2021 21:51
@joaquinelio
Copy link
Copy Markdown
Member Author

probados online ok

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants